Skip to content

grpc-web: remove envoy.reloadable_features.grpc_web_fix_non_proto_encoded_response_handling#18620

Merged
lizan merged 3 commits intoenvoyproxy:mainfrom
dio:fix-18436
Nov 1, 2021
Merged

grpc-web: remove envoy.reloadable_features.grpc_web_fix_non_proto_encoded_response_handling#18620
lizan merged 3 commits intoenvoyproxy:mainfrom
dio:fix-18436

Conversation

@dio
Copy link
Copy Markdown
Member

@dio dio commented Oct 14, 2021

Commit Message: This patch removes
envoy.reloadable_features.grpc_web_fix_non_proto_encoded_response_handling
and relevant legacy code paths.
Risk Level: Low
Testing: Updated
Docs Changes: N/A
Release Notes: Added
Platform-Specific Features: N/A
Fixes #18436

Signed-off-by: Dhi Aurrahman dio@rockybars.com

…oded_response_handling

This patch removes
envoy.reloadable_features.grpc_web_fix_non_proto_encoded_response_handling
and relevant legacy code paths.

Fixes #18436

Signed-off-by: Dhi Aurrahman <dio@rockybars.com>
@dio dio requested a review from lizan as a code owner October 14, 2021 01:46
@dio
Copy link
Copy Markdown
Member Author

dio commented Oct 14, 2021

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #18620 (comment) was created by @dio.

see: more, trace.

@lizan
Copy link
Copy Markdown
Member

lizan commented Oct 27, 2021

@dio can you resolve conflicts?

Signed-off-by: Dhi Aurrahman <dio@rockybars.com>
@lizan lizan merged commit 3a87cd0 into envoyproxy:main Nov 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

envoy.reloadable_features.grpc_web_fix_non_proto_encoded_response_handling deprecation

2 participants